Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Android] do not call setHyphenationFrequency on AndroidSdk < 23 #29258

Closed

Conversation

fabOnReact
Copy link
Contributor

@fabOnReact fabOnReact commented Jul 1, 2020

Summary

@JoshuaGross This issue fixes #28279 as discussed in #29157 (comment)
Avoid calling setHyphenationFrequency on Android Sdk < 23.

Changelog

[Android] [Fixed] - do not call setHyphenationFrequency on AndroidSdk < 23

Test Plan

BEFORE AFTER

The warning displayed with adb logcat | grep -P "ReactTextAnchorViewManager"

image

I remain available to do improvements. Thanks a lot. Fabrizio.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jul 1, 2020
@react-native-bot react-native-bot added Bug Platform: Android Android applications. labels Jul 1, 2020
@analysis-bot
Copy link

Platform Engine Arch Size (bytes) Diff
ios - universal n/a --

Base commit: e261f02

@analysis-bot
Copy link

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 6,763,081 146
android hermes armeabi-v7a 6,426,157 160
android hermes x86 7,150,839 164
android hermes x86_64 7,040,770 156
android jsc arm64-v8a 8,935,434 149
android jsc armeabi-v7a 8,590,898 154
android jsc x86 8,766,294 143
android jsc x86_64 9,341,854 150

Base commit: e261f02

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mdvacca has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@mdvacca
Copy link
Contributor

mdvacca commented Jul 1, 2020

@fabriziobertoglio1987 can you update the documentation with this change as well?

1 similar comment
@mdvacca
Copy link
Contributor

mdvacca commented Jul 1, 2020

@fabriziobertoglio1987 can you update the documentation with this change as well?

Copy link
Contributor

@mdvacca mdvacca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @fabriziobertoglio1987 in 7d8aeb4.

When will my fix make it into a release? | Upcoming Releases

@fabOnReact
Copy link
Contributor Author

fabOnReact commented Jul 3, 2020

Thanks a lot! facebook/react-native-website#2007

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. Platform: Android Android applications.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Android ignores unicode soft hyphen (u00AD)
5 participants